배포가 Alloy 프로비저닝을 포함하도록 - #39
Merged
Merged
Conversation
- 이 박스의 Alloy 설정이 배포와 무관하게 옛 상태로 남던 문제. provision-observability.sh 는 필요한 일을 이미 다 하고 있었지만 배포가 호출하지 않아, 사람이 박스에서 수동 실행할 때만 반영됐다 - 실측 근거: infra#44 로 config 를 감축(수집 60s·누수 차단·collector 41→12)한 뒤 이 박스를 재배포해도 collector 41 그대로였다. 같은 변경이 적용된 core dev 박스는 12 - core 만 deploy 에 provision 단계가 있어 생긴 비대칭이다 - 앱 배포 앞에 둔다: core 와 순서를 맞추고, 관측이 깨진 채 앱만 올라가는 상태를 막는다 - 스크립트가 자기 완결적이라(공용 블록 fetch + 박스가 SSM 직접 조회) 러너가 넘길 인자·secret 이 없다
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Situation
provision-observability.sh는 필요한 일(infra 공용 블록 fetch → SSM 자격 로드 → 자기 박스 값으로 호출)을 이미 전부 하지만, 배포 워크플로가 그것을 호출하지 않아 사람이 박스에서 수동 실행할 때만 반영됐다.deploy.yml이provision-runtime.sh로 Alloy 를 갱신core 만 배포에 프로비저닝 단계가 있어 생긴 비대칭이고, 그 결과 공용 config 를 고쳐도 5개 박스 중 2개(core dev·prod)에만 퍼졌다.
Task
Action
deploy.yml에 기존 스크립트를 실행하는 단계를 추가했다.Upload blocks의 source 에provision-observability.sh를 더해 박스로 올린다.raw.githubusercontent에서 직접 fetch, 자격은 박스가 SSM 공유 경로에서 직접 조회) 러너가 넘길 인자나 secret 이 없다.검토했으나 택하지 않은 안
SSM 로드 로직이 세 곳(core
provision-runtime.sh4절, extractor·renderer 각자의provision-observability.sh)에 중복이라, 이를 infra 공용 블록 하나로 통합하는 안을 먼저 만들었다(TeamPiKi/infra#46). 그러나 각 repo 의 스크립트가 이미 잘 동작하고 공용 블록을 fetch 해 쓰는 구조라 SSOT 도 어느 정도 지켜지고 있었다. 잘 도는 코드 세 벌을 건드리는 비용이 이득보다 커 폐기하고, 실제 결함(배포가 호출하지 않음)만 최소 변경으로 고쳤다. 중복 정리는 필요해질 때 별건으로 둔다.Result
node_scrape_collector_success시리즈 수가 41 → 12 로 떨어지는지,veth·/snap시계열이 사라지는지로 검증할 수 있다.연관 이슈